Skip to content

Feature/bidi 20250301 #2203

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 1, 2025
Merged

Conversation

alaahong
Copy link
Member

@alaahong alaahong commented Mar 1, 2025

User description

Thanks for contributing to the Selenium site and documentation!
A PR well described will help maintainers to review and merge it quickly

Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, and help reviewers by making them as simple and short as possible.

Description

website_and_docs/content/documentation/webdriver/bidi/_index.zh-cn.md

Motivation and Context

Update Bidi

Types of changes

  • Change to the site (I have double-checked the Netlify deployment, and my changes look good)
  • Code example added (and I also added the example to all translated languages)
  • Improved translation
  • Added new translation (and I also added a notice to each document missing translation)

Checklist

  • I have read the contributing document.
  • I have used hugo to render the site/docs locally and I am sure it works.

PR Type

Documentation


Description

  • Improved Chinese translations for BiDi and IE documentation.

  • Updated BiDi documentation to enhance clarity and accuracy.

  • Revised IE-specific documentation with better formatting and details.

  • Added examples and explanations for IE driver options and logging.


Changes walkthrough 📝

Relevant files
Documentation
_index.zh-cn.md
Enhanced BiDi documentation with Chinese translation         

website_and_docs/content/documentation/webdriver/bidi/_index.zh-cn.md

  • Translated BiDi documentation into Chinese.
  • Improved clarity and formatting of the content.
  • Added explanations for WebDriver BiDi protocol features.
  • Updated examples for enabling BiDi in Selenium.
  • +38/-26 
    internet_explorer.zh-cn.md
    Updated IE-specific documentation with Chinese translation

    website_and_docs/content/documentation/webdriver/browsers/internet_explorer.zh-cn.md

  • Translated IE-specific documentation into Chinese.
  • Improved formatting and added detailed explanations.
  • Updated examples for IE driver options and logging.
  • Clarified IE compatibility mode and driver capabilities.
  • +53/-36 

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • Copy link

    netlify bot commented Mar 1, 2025

    👷 Deploy request for selenium-dev pending review.

    Visit the deploys page to approve it

    Name Link
    🔨 Latest commit 1784106

    Copy link
    Contributor

    qodo-merge-pro bot commented Mar 1, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Translation Consistency

    Verify that technical terms like 'WebDriver BiDi', 'WebSocket', and 'CDP' are consistently translated and maintain their technical meaning in Chinese

    目前, [Chrome DevTools Protocol] (CDP) 可以有限地提供这种功能, 
    但为了解决它的一些缺点, 
    Selenium 团队与主要浏览器供应商一起创建了新的 [WebDriver BiDi 协议](https://w3c.github.io/webdriver-bidi/).
    该规范旨在创建一个稳定的跨浏览器 API, 
    利用双向通信增强浏览器自动化和测试功能、
    包括通过 WebSockets 从用户代理到控制软件的流式事件. 

    Copy link
    Contributor

    qodo-merge-pro bot commented Mar 1, 2025

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    General
    Improve clarity of Chinese title

    The Chinese translation of "双向功能" is too literal and may be unclear. Consider
    using a more descriptive title that better explains the BiDi concept, such as
    "双向通信功能" or "双向协议功能".

    website_and_docs/content/documentation/webdriver/bidi/_index.zh-cn.md [1-3]

     ---
    -title: "双向功能"
    +title: "双向通信功能"
     linkTitle: "BiDi"

    [Suggestion has been applied]

    Suggestion importance[1-10]: 5

    __

    Why: The suggestion improves clarity by making the Chinese title more descriptive and specific to the BiDi communication concept, which would help Chinese readers better understand the topic.

    Low
    Use more specific section heading

    The Chinese translation of "特别注意事项" is too generic. Use a more specific heading
    like "重要限制说明" to better convey the section's content about limitations and
    special considerations.

    website_and_docs/content/documentation/webdriver/browsers/internet_explorer.zh-cn.md [15]

    -## 特别注意事项
    +## 重要限制说明
    • Apply this suggestion
    Suggestion importance[1-10]: 4

    __

    Why: The suggested heading "重要限制说明" is more specific and better reflects the section's content about IE driver limitations and considerations, improving document navigation.

    Low
    • Update

    @alaahong alaahong merged commit d4164e6 into SeleniumHQ:trunk Mar 1, 2025
    3 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant